Skip to content

Conversation

@khassel
Copy link
Collaborator

@khassel khassel commented May 15, 2025

This is a draft PR which we discussed in the forum.

I'm not really sure that it worked with auth for all repos so please review the job log.

The result looks good so far, it is published here. The MMM-HusqvarnaAutomower repo has now an image and the correct license.

Job needed 18min. compared to 11:30 before (which is still good).

@khassel khassel force-pushed the auth branch 3 times, most recently from 5410f17 to e427891 Compare May 15, 2025 21:38
@KristjanESPERANTO
Copy link
Collaborator

Nice! I can't see an issue in the logs.

We can merge as far as I'm concerned and I'd do a bit fine-tuning afterwards so that this line in the logs, for example, doesn't look so contradictory:

GitHub data update completed. queryCount: 2492 maxQueryCount: 58 results: 1246 modules: 1259

@khassel khassel force-pushed the auth branch 3 times, most recently from 08f6899 to b4f1b0c Compare May 16, 2025 17:55
@khassel
Copy link
Collaborator Author

khassel commented May 16, 2025

GitHub data update completed. queryCount: 2492 maxQueryCount: 58 results: 1246 modules: 1259

Quick solution for me was to set maxQueryCount to 0 if it is lower than queryCount ...

Other point: With a full update per pipeline run I think it is sufficient to run the job only once a day.

If this is o.k. for you I would update the sentence in the README.md here:

Twice a day the new list will be updated based on the original list.

@KristjanESPERANTO
Copy link
Collaborator

If this is o.k. for you I would update the sentence in the README.md ...

Yes, once a day should also be enough.

@sdetweil
Copy link

can you estimate when it will end, in what timezone?

@khassel
Copy link
Collaborator Author

khassel commented May 16, 2025

can you estimate when it will end, in what timezone?

atm the build is done in gitlab, there it runs ~20min. and I can set time and timezone as we want.

@KristjanESPERANTO
Copy link
Collaborator

The timing is not important to me. I think that since we have users all over the world, any time is okay. Sam, do you have a wish?

@sdetweil
Copy link

I don't care when or where(tz), just don't want to pull the file if it hasn't been updated today
maybe a last_updated file with that info

@khassel khassel marked this pull request as ready for review May 16, 2025 20:48
@KristjanESPERANTO
Copy link
Collaborator

Last updated you can see here: https://modules.magicmirror.builders/data/stats.json

@khassel khassel requested a review from KristjanESPERANTO May 16, 2025 20:49
@sdetweil
Copy link

right, but have to drag down the entire file to get 1 line. dont want to do this 100 times a day for nothing.

@khassel
Copy link
Collaborator Author

khassel commented May 16, 2025

current setting:

grafik

so let us know which TZ and schedule time you want and you can add 30min

@sdetweil
Copy link

30 5 * * *

existing tz is fine

Copy link
Collaborator

@KristjanESPERANTO KristjanESPERANTO left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! 😃

@KristjanESPERANTO KristjanESPERANTO merged commit fff1a33 into MagicMirrorOrg:main May 16, 2025
2 checks passed
@khassel khassel deleted the auth branch May 16, 2025 21:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants